The createZip function had incomplete error handling: both 'close' and 'error' events could fire simultaneously causing double resolution, and errors from intermediate streams in the pipe chain were not handled, risking unhandled rejections.

Changes:

• Added settled flag to prevent double promise resolution/rejection
• Attached explicit error handlers to all streams in the pipe chain (zipFile.outputStream, progressStream, outputStream)
• Implemented cleanup via destroy() on all streams when errors occur

Before:

zipFile.outputStream
  .pipe(progressStream)
  .pipe(outputStream)
  .on('close', () => resolve())
  .on('error', reject)

After:

const handleError = (error: Error) => {
  if (settled) return
  settled = true
  zipFile.outputStream.destroy()
  progressStream.destroy()
  outputStream.destroy()
  reject(error)
}

zipFile.outputStream.on('error', handleError)
progressStream.on('error', handleError)
outputStream.on('error', handleError)
outputStream.on('close', handleSuccess)

zipFile.outputStream
  .pipe(progressStream)
  .pipe(outputStream)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.